Skip to content
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.

Additional Pentest Doc Updates #1178

Merged
merged 2 commits into from
Oct 2, 2023
Merged

Additional Pentest Doc Updates #1178

merged 2 commits into from
Oct 2, 2023

Conversation

bwillis
Copy link
Contributor

@bwillis bwillis commented Sep 29, 2023

No description provided.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ This pull request was sent to the PullRequest network.


Check the status or cancel PullRequest code review here.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PullRequest Breakdown

Reviewable lines of change

+ 130
- 59

89% Markdown
11% YAML
<1% Other

Type of change

Feature - These changes are adding a new feature or improvement to existing code.
1 Message
📚 It looks like the description for this pull request is either blank or very short. Adding a high-level summary will help our reviewers provide better feedback. Feel free to include questions for PullRequest reviewers and make specific feedback requests.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall the change looks fine. I noticed and pointed out several minor issues within the changes, such as using Markdown within HTML blocks

Image of Xiaoyong W Xiaoyong W


Reviewed with ❤️ by PullRequest

docs/organizations/pentest-phases.md Outdated Show resolved Hide resolved
docs/organizations/pentest-phases.md Outdated Show resolved Hide resolved
docs/organizations/pentest-phases.md Outdated Show resolved Hide resolved
docs/organizations/retesting-pentest.md Outdated Show resolved Hide resolved
docs/organizations/retesting-pentest.md Outdated Show resolved Hide resolved
Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PullRequest reviewed the updates made to #1178 up until the latest commit (7e0e58c). No further issues were found.

Reviewed by:

Image of Xiaoyong W Xiaoyong W

@jessiwright jessiwright merged commit 8f4eb27 into master Oct 2, 2023
3 checks passed
@bwillis bwillis deleted the pentest-docs-updates branch October 2, 2023 16:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants